Skip to content

Update context menu logic and implement shortcuts#562

Merged
BenjaTK merged 4 commits intogaea-godot:2.0from
Zehir:better_menu
Feb 16, 2026
Merged

Update context menu logic and implement shortcuts#562
BenjaTK merged 4 commits intogaea-godot:2.0from
Zehir:better_menu

Conversation

@Zehir
Copy link
Collaborator

@Zehir Zehir commented Jan 7, 2026

This PR changes some menus and add shortcut to them.

The logic for the Filesystem menus have been moved to the FileSystemContainer script to be used by both menus;
imageimage

The new Edit popup use the node context menu actions
imageimage

The shortcuts use editor settings ones for many editor shortcuts, so if the user change them in the Editor Setting the label will be updated. If the user is using Godot 4.6+ the shortcuts will be added to the Editor Setting and he will be able to change them. In Godot 4.5 they are fixed shortcuts.

Also some shortcuts have been added to the file system :

  • New Graph: Ctrl + N (classic new file)
  • Open Graph: Ctrl + O (classic open file)
  • Save Graph: Ctrl + Alt + S (because for some reason it what is used in script editor, will save only the selected graph but Ctrl+S still save the complete project so also the opened graph)
  • Close Graph: Ctrl + W (close tab from browsers and also in script editor)

Also some shortcuts have been added to the file Graph Edit :

  • Add node : Ctrl + T (inspiration of new tab on browsers)
  • Copy, Cut, Paste, Duplicate, Delete was already working but not displayed to the user
  • Select all nodes: Ctrl + A (classic)
  • Group nodes in frame: Ctrl + G (why not, there is a Ctrl + G shortcut in the editor called "Editor/Group Selected Node(s) but was not able to find what this does")
image

@Zehir Zehir marked this pull request as ready for review January 8, 2026 19:27
@Zehir Zehir requested a review from cullumi January 8, 2026 19:27
@BenjaTK BenjaTK added ✨ enhancement New feature or request topic: editor Changes to the UI or UX of the editor. labels Jan 11, 2026
@BenjaTK BenjaTK changed the title Update graph editor menus and add shortcuts Update context menu logic and implement shortcuts Jan 11, 2026
@BenjaTK
Copy link
Collaborator

BenjaTK commented Feb 1, 2026

4.6 added an easy way to add configurable shortcuts, maybe want to look into that?

@BenjaTK
Copy link
Collaborator

BenjaTK commented Feb 1, 2026

nvm I'm dumb

@BenjaTK
Copy link
Collaborator

BenjaTK commented Feb 1, 2026

still, the 4.5 logic can be removed since we're updating to 4.6 from now on.

@Zehir
Copy link
Collaborator Author

Zehir commented Feb 1, 2026

still, the 4.5 logic can be removed since we're updating to 4.6 from now on.

I have removed the 4.6 check

@Zehir Zehir requested a review from BenjaTK February 1, 2026 15:12
@cullumi cullumi added this to the v2.0.0-stable milestone Feb 9, 2026
Copy link
Collaborator

@cullumi cullumi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am unable to get any of this to show up on either Godot 4.5 or 4.6. This is the second PR where I just don't see the new features showing up anywhere so there's a chance something weird is happening on my end.

Psych, I opened the wrong branch, lol

cullumi
cullumi previously requested changes Feb 9, 2026
Copy link
Collaborator

@cullumi cullumi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that only some of the Gaea dock's shortcuts are actually added to the Editor Settings, or is this because the most "standard" ones are just reused from the common shortcuts? (i.e. Delete Selection : Delete)

Additionally, the way the shortcuts are displayed is not working as expected, see below:

@Zehir
Copy link
Collaborator Author

Zehir commented Feb 11, 2026

It seems that only some of the Gaea dock's shortcuts are actually added to the Editor Settings, or is this because the most "standard" ones are just reused from the common shortcuts? (i.e. Delete Selection : Delete)

Additionally, the way the shortcuts are displayed is not working as expected, see below:

yes I reuse some of the common shortcuts and some are built-in actions from the graph edit directly. I just display them so the user know about them

@Zehir Zehir requested a review from BenjaTK February 16, 2026 19:08
Copy link
Collaborator

@BenjaTK BenjaTK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BenjaTK BenjaTK merged commit f447802 into gaea-godot:2.0 Feb 16, 2026
5 checks passed
@Zehir Zehir deleted the better_menu branch February 16, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ enhancement New feature or request topic: editor Changes to the UI or UX of the editor.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants